Skip to content
This repository has been archived by the owner on Dec 27, 2019. It is now read-only.

Convert to gem #11

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Convert to gem #11

wants to merge 5 commits into from

Conversation

jwworth
Copy link

@jwworth jwworth commented Jun 1, 2016

Close issue #8

This change converts this project from a Jekyll plugin to a gem. Unfinished tasks include adding author and email to the gemspec, deploying the first build to Rubygems, and testing.

@@ -0,0 +1,12 @@
Gem::Specification.new do |s|
s.name = 'jekyll_get'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't it be jekyll-get?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm suggesting renaming the main module of this gem to JekyllGet (was Jekyll_Get). Classes and modules in Ruby use MixedCase and have no underscores. Naming the gem jekyll_get was a result of this decision, per Rubygems naming recommendations: http://guides.rubygems.org/name-your-gem/

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation.

@mdxprograms
Copy link

Can this be merged please?

@KrzysztofMadejski
Copy link

@afeld @cat4 maybe you should add more owners to this repo so someone can maintain it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants